Skip to content
This repository was archived by the owner on Dec 19, 2018. It is now read-only.

Conversation

@ajaybhargavb
Copy link
Contributor

@ghost ghost added the cla-not-required label Mar 17, 2015
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove 😺

@dougbu
Copy link
Contributor

dougbu commented Mar 17, 2015

⌚ just one more bit of cleanup

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should be able to remove the SyntaxConstants.CSharp.HelperKeyword

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed.

@NTaylorMullen
Copy link

@ajaybhargavb
Copy link
Contributor Author

Updated.

@dougbu
Copy link
Contributor

dougbu commented Mar 17, 2015

:shipit: once @NTaylorMullen is moderately pleased

@NTaylorMullen
Copy link

Ensure this doesn't break anything in MVC then :shipit:, if it does we can do a quick react PR.

@ajaybhargavb ajaybhargavb merged commit b3c6097 into dev Mar 17, 2015
@ajaybhargavb ajaybhargavb deleted the remove-helper branch March 17, 2015 21:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants